Probably not ready to be pulled in just yet... I'd rather we just had IPython as a dependency and then we could move these definitions into plotly.py. Thoughts?
@etpinard, @chriddyp

👍 for doing this.

I have mixed feelings about making IPython a dependency for plotly.

Yes plotly is at its best inside IPython notebooks (so it might a good idea to force users to use IPython).
But then again, I feel this could push some users away.

ok, yeah, i think i'm leaning that way too. just wanted another opinion on it. we can probably do something like this:

if _ipython_imported:
    class PlotlyFrame(ipython.core.display.HTML):
        # blah blah

That way it's private (well... not there at all really) if folks don't have ipython, it's not defined dynamically, which is not really an appropriate solution here, and we can put it whereverz. it's sort of like this: https://github.com/plotly/python-api/blob/master/plotly/tools.py#L27-31

Seem alright?

@theengineear good call.

If you have matplotlib installed already then plotly has something for you, but if you don't: no worries.
And similarly, if you have ipython installed then plotly has also something for you.

💃

luv it - def shouldn't be a requirement!

On Mon, Aug 18, 2014 at 4:18 PM, Étienne Tétreault-Pinard <
[email protected]> wrote:

@theengineear https://github.com/theengineear good call.

If you have matplotlib installed already then plotly has something for
you.
And similarly, if your have ipython installed then plotly has also
something for you.

[image: 💃]

—
Reply to this email directly or view it on GitHub
#107 (comment).

k, pretty sure everyone is onboard. merging.